-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switching to QED-it orchard Rust crate #3
Conversation
* Added .circleci/config.yml
* Added .circleci/config.yml
@@ -91,3 +91,4 @@ zcash_history = { git = "https://github.com/zcash/librustzcash.git", rev = "d5c5 | |||
zcash_note_encryption = { git = "https://github.com/zcash/librustzcash.git", rev = "d5c5f048947d211c2fbef23ce986b329b91d1aa5" } | |||
zcash_primitives = { git = "https://github.com/zcash/librustzcash.git", rev = "d5c5f048947d211c2fbef23ce986b329b91d1aa5" } | |||
zcash_proofs = { git = "https://github.com/zcash/librustzcash.git", rev = "d5c5f048947d211c2fbef23ce986b329b91d1aa5" } | |||
orchard = { git = "https://github.com/QED-it/orchard.git", rev = "6d2834bfd3230dc6a52b5b0c99e149f92455de69"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this has changed already zcash@5fc38dd#diff-2e9d962a08321605940b5a657135052fbcef87b5e360662bb527c96d9a615542L86 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The referenced changes are because we focused on (and succeeded in) ensuring that zcashd 5.0.0
only depended on released crate versions. Adding this patch directive is the correct way for QEDIT to experiment with their patches to the orchard
crate (and is precisely what we do for longer-lived development cycles).
This PR allows us to use the QED-it orchard crate.
Building locally would require using
CONFIGURE_FLAGS=--enable-online-rust ./zcutil/build.sh -j8
, as that allows using the dependencies specified in Cargo.toml instead of those in Cargo.lockNote that when the orchard crate is updated, it will be required to update the Cargo.toml file to reflect any newer commits.